Skip to content

[build-tools] Support individual ASC API keys in upload_to_asc - #4247

Draft
sswrk wants to merge 1 commit into
mainfrom
szymonswierk/eng-21475-build-tools-upload-to-asc-individual-keys
Draft

[build-tools] Support individual ASC API keys in upload_to_asc#4247
sswrk wants to merge 1 commit into
mainfrom
szymonswierk/eng-21475-build-tools-upload-to-asc-individual-keys

Conversation

@sswrk

@sswrk sswrk commented Aug 21, 2026

Copy link
Copy Markdown
Member

Why

ASC submissions don't need a team API key. Individual key works too.

The eas/upload-to-asc function requires issuer_id in the API key json currently.

How

  • Make issuer_id optional in the zod schema for the API key in eas/upload-to-asc.
  • When issuer_id nullish, iss is not set during the token creation, but sub is set to user instead.

Test Plan

CI passes.

@linear-code

linear-code Bot commented Aug 21, 2026

Copy link
Copy Markdown

ENG-21475

@sswrk
sswrk force-pushed the szymonswierk/eng-21475-build-tools-upload-to-asc-individual-keys branch from 691fdf7 to d5e280f Compare August 21, 2026 17:17
@codecov

codecov Bot commented Aug 21, 2026

Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 0% with 4 lines in your changes missing coverage. Please review.
✅ Project coverage is 64.41%. Comparing base (51ba719) to head (d9ccc69).
⚠️ Report is 2 commits behind head on main.

Files with missing lines Patch % Lines
...ges/build-tools/src/steps/functions/uploadToAsc.ts 0.00% 4 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main    #4247      +/-   ##
==========================================
+ Coverage   64.41%   64.41%   +0.01%     
==========================================
  Files        1048     1048              
  Lines       48425    48428       +3     
  Branches    10203    10205       +2     
==========================================
+ Hits        31188    31192       +4     
+ Misses      17121    17120       -1     
  Partials      116      116              

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@sswrk
sswrk force-pushed the szymonswierk/eng-21475-build-tools-upload-to-asc-individual-keys branch from d5e280f to d9ccc69 Compare August 28, 2026 14:02
@sswrk sswrk added the no changelog PR that doesn't require a changelog entry label Aug 28, 2026
@github-actions

Copy link
Copy Markdown

⏩ The changelog entry check has been skipped since the "no changelog" label is present.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

no changelog PR that doesn't require a changelog entry

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant